Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

스페이스 추가 화면 Jetpack Compose로 마이그레이션 하기 #324

Merged
merged 16 commits into from
Mar 9, 2024
Merged

스페이스 추가 화면 Jetpack Compose로 마이그레이션 하기 #324

merged 16 commits into from
Mar 9, 2024

Conversation

jaehan4707
Copy link
Member

관련 이슈

작업한 내용

  • 기존 화면과 동일한 UI를 XML이 아닌 Compose로 마이그레이션
image

To Reviewers

상준님이 마이그레이션 하셔서 저도 도전해봤습니다.
해보고 느낀점은 코드의 양이 정말 줄어든다는 점인것 같습니다.
기존 Activitiy의 코드와 Xml의 코드를 하나로 합치고, Xml에서의 방식이 아닌 온전한 Kotlin으로 UI를 구현할 수 있다는 점은 엄청난 
메리트인것 같습니다!!

@jaehan4707
refactor(#322): AddSpaceUI 구현
c5810a1
@jaehan4707
refactor(#322):Component 분리
cc13bda
@jaehan4707
refactor(#322):TextField hint, 텍스트 입력 구현
b042030
@jaehan4707
refactor(#322): 뒤로 가기 구현
fea073d
@jaehan4707
refactor(#322):photoPicker 구현
f7bc2ea
@jaehan4707
refactor(#322): 스페이스 추가 구현(그 외 이벤트 처리 미완성)
619c876
@jaehan4707
refactor(#322): imageLauncher 분리, Preview 활성화
cfae3e9
@jaehan4707
design(#322): placeHolder image 추가
17897ab
@jaehan4707
refactor(#322): placeHolder,Error 이미지 추가
6d3a301
@jaehan4707
refactor(#322): Scaffold 적용
db0ab7c
@jaehan4707
refactor(#322): TextFiled 글자수 카운트 추가
60d0807
@jaehan4707
refactor(#322): 스페이스 이름에 따른 버튼 enable 처리, enable에 따른 색깔 처리
1ce76da
@jaehan4707
refactor(#322): uiEvent 연결
5ec655f
@jaehan4707
style(#322): ktlint 적용
639b86d
@jaehan4707
chore(#322): 기존 AddSpaceActivity 코드 제
3f31545
@jaehan4707 jaehan4707 added the 💪Refactor 코드 리팩토링 label Mar 3, 2024
@jaehan4707 jaehan4707 self-assigned this Mar 3, 2024
@github-actions github-actions bot added 📱AOS 안드로이드 작업 🎨UI UI 작업 labels Mar 3, 2024
@jaehan4707 jaehan4707 added the 💡Feature 기능 개발 label Mar 3, 2024
yang1318
yang1318 approved these changes Mar 7, 2024
View reviewed changes
Copy link
Collaborator

@yang1318 yang1318 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

오옹.. 고생하셨습니다! 정진하시는 모습이 보기 좋아요 ✨
그런데 대문자로 시작하는 함수이름이 ktlint에 많이 걸렸네요 ㅠㅠ
상준님 PR에서는 걸리지 않았던 것 같은데, 혹시 안걸리게 할 수 있는 다른 방법은 없을까요?

hegleB
hegleB approved these changes Mar 7, 2024
View reviewed changes
Copy link
Member

@hegleB hegleB left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨습니당 ~ compose 사용해보니 이제 XML로 못 돌아 갈거 같아요 🤣

@hegleB hegleB merged commit e8bdab0 into AOS-develop Mar 9, 2024
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yang1318 yang1318 yang1318 approved these changes

@hegleB hegleB hegleB approved these changes

Assignees

@jaehan4707 jaehan4707

Labels
📱AOS 안드로이드 작업 💡Feature 기능 개발 💪Refactor 코드 리팩토링 🎨UI UI 작업
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jaehan4707 @hegleB @yang1318